home *** CD-ROM | disk | FTP | other *** search
/ NetNews Offline 2 / NetNews Offline Volume 2.iso / news / comp / sys / amiga / programmer / 2960 < prev    next >
Encoding:
Internet Message Format  |  1996-08-05  |  1.6 KB
  1. Path: hearst.acc.Virginia.EDU!adastra!mbs
  2. Newsgroups: comp.sys.amiga.programmer
  3. From: mbs@adastra.cvl.va.us (Michael B. Smith)
  4. Subject: Re: YASL (was: AsynchIO (was: fastest file read method ??))
  5. References:  <3116c08e@lls.se>
  6. X-NewsReader: GRn 3.0a February 1, 1996
  7. MIME-Version: 1.0
  8. Content-Type: text/plain; charset=iso-8859-1
  9. Content-Transfer-Encoding: 8bit
  10. Message-ID: <mbs.4892@adastra.cvl.va.us>
  11. Date: Wed, 7 Feb 96 07:06:57 EDT
  12. Organization: Only if you insist...
  13.  
  14. In article <3116c08e@lls.se> cmh@lls.se (Magnus Holmgren) writes:
  15. > In a message of 29 Jan 96, Michael wrote to :
  16. >
  17. >  <AsyncIO>
  18. >
  19. >  MBS> Actually, SeekAsync() is still badly buggy, in every version I've
  20. >  MBS> yet seen.
  21. >
  22. > Even release 7? I have fixed a couple of bugs, and I don't know of any
  23. > holes at the moment, but that doesn't mean that there aren't any. ;)
  24. >
  25. > If you could report them (preferably with a compilable example in C), I'd
  26. > have a chance in fixing them. ;)
  27.  
  28. I've not tried a version of yours since 4 (which I had already said in an
  29. article early in this thread).
  30.  
  31. The base asyncio code (i.e., Martin's) has a real problem with early buffer
  32. spills. Up through v4 of your stuff, while it didn't appear to be reading
  33. extra blocks, the data point was seriously offbase.
  34.  
  35. My "example" is a mailreader, and is several thousand lines of source. To
  36. prove it wasn't my code, I re-did it using dos.library buffered I/O at one
  37. point. That worked. The original seekasync() worked (slowly). Yours gave
  38. me the wrong data.
  39.  
  40. I'll try to find time to look at v7, but it'll be a week or so.
  41. --
  42.   //   Michael B. Smith
  43. \X/    mbs@adastra.cvl.va.us
  44.